Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove support for graal #416

Merged
merged 1 commit into from
Oct 28, 2024
Merged

Remove support for graal #416

merged 1 commit into from
Oct 28, 2024

Conversation

froth
Copy link
Member

@froth froth commented Oct 23, 2024

see #414

@froth
Copy link
Member Author

froth commented Oct 23, 2024

/home/runner/work/_temp/75ea71be-cd87-4c88-97da-3f552e83deaf.sh: line 3: gu: command not found

Maybe we just drop graal support for this template. My motivation to debug this is not particularly high.

@rossabaker
Copy link
Member

I'm in favor of dropping it. I don't imagine there are many users who go down this path.

@froth froth changed the title Update graalvm image to fix build. Remove support for graal Oct 25, 2024
@froth
Copy link
Member Author

froth commented Oct 25, 2024

@rossabaker done. Makes the build a lot faster, which is a nice bonus.

@froth froth mentioned this pull request Oct 25, 2024
Merged
@rossabaker rossabaker merged commit d156e96 into 0.23 Oct 28, 2024
10 checks passed
@froth froth deleted the update-graal branch October 28, 2024 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danicheg danicheg danicheg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@froth @rossabaker @danicheg